Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

Enabled more linters #150

Merged
merged 7 commits into from
Oct 28, 2022

Conversation

andresuribe87
Copy link
Contributor

As a follow up to #149, this PR enables a number of additional linters, and fixes the corresponding issues found.

@codecov-commenter
Copy link

codecov-commenter commented Oct 28, 2022

Codecov Report

Merging #150 (bebe637) into main (8905a12) will not change coverage.
The diff coverage is 22.22%.

@@           Coverage Diff           @@
##             main     #150   +/-   ##
=======================================
  Coverage   27.79%   27.79%           
=======================================
  Files          20       20           
  Lines        1558     1558           
=======================================
  Hits          433      433           
  Misses       1051     1051           
  Partials       74       74           
Impacted Files Coverage Δ
pkg/server/router/credential.go 5.40% <0.00%> (ø)
pkg/server/router/dwn.go 17.77% <ø> (ø)
pkg/dwn/dwn.go 61.53% <66.66%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@decentralgabe decentralgabe merged commit 3603ace into TBD54566975:main Oct 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants